Skip to content

[fix](Nereids) move tables from connect context to statement context (#44568)#45216

Closed
LiBinfeng-01 wants to merge 1 commit intoapache:branch-2.1from
LiBinfeng-01:fix_tables_in_connectcontext_2.1
Closed

[fix](Nereids) move tables from connect context to statement context (#44568)#45216
LiBinfeng-01 wants to merge 1 commit intoapache:branch-2.1from
LiBinfeng-01:fix_tables_in_connectcontext_2.1

Conversation

@LiBinfeng-01
Copy link
Contributor

@LiBinfeng-01 LiBinfeng-01 commented Dec 10, 2024 •
edited
Loading

pick: (#44568)

Problem Summary:
When using tables in connect context, it would keep on memory in next run in the same session, but it should not be in memory when running next sql statement

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason

  • Behavior changed:

    • No.
    • Yes.

  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@doris-robot
Copy link

doris-robot commented Dec 10, 2024

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@LiBinfeng-01
Copy link
Contributor Author

LiBinfeng-01 commented Dec 10, 2024

run buildall

@yiguolei
[fix](Nereids) move tables from connect context to statement context (a…
107f779 
…pache#44568)

Problem Summary:
When using tables in connect context, it would keep on memory in next
run in the same session, but it should not be in memory when running
next sql statement
@yiguolei yiguolei force-pushed the fix_tables_in_connectcontext_2.1 branch from 20a6f4c to 107f779 Compare December 18, 2024 15:00
@yiguolei
Copy link
Contributor

yiguolei commented Dec 18, 2024

run buildall

@morrySnow
Copy link
Contributor

morrySnow commented Dec 24, 2024

already processed by #45679

@morrySnow morrySnow closed this Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@LiBinfeng-01 @doris-robot @yiguolei @morrySnow